Skip to content

Comments

chore(deps): remove unused frontend dependencies#3152

Merged
arkid15r merged 1 commit intoOWASP:mainfrom
SuyashJain17:chore/remove-unused-frontend-deps
Jan 2, 2026
Merged

chore(deps): remove unused frontend dependencies#3152
arkid15r merged 1 commit intoOWASP:mainfrom
SuyashJain17:chore/remove-unused-frontend-deps

Conversation

@SuyashJain17
Copy link
Contributor

Proposed change

Resolves #3142

Summary

This PR removes unused frontend dependencies after verification.

Details

  • Removed rxjs from frontend/package.json as it is not imported or used anywhere in the codebase or tests.
  • Removed @types/react-gtm-module after confirming there are no references to Google Tag Manager and the runtime package is not present.
  • Updated pnpm-lock.yaml accordingly using the repository-specified pnpm version.

This change is limited to dependency cleanup and does not affect application behavior.

Checklist

  • Required: I read and followed the contributing guidelines
  • Required: I ran dependency installation locally and verified the frontend setup after the change
  • I used AI for guidance during investigation and verification; all changes were reviewed and applied manually

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 2, 2026

Summary by CodeRabbit

  • Chores
    • Removed unused dependencies from project configuration to streamline development and reduce maintenance requirements.

✏️ Tip: You can customize this high-level summary in your review settings.

Walkthrough

Removed two unused dependencies from the frontend package configuration: rxjs from production dependencies and @types/react-gtm-module from development dependencies. No code or exported declarations were modified.

Changes

Cohort / File(s) Summary
Dependency cleanup
frontend/package.json
Removed unused rxjs dependency and unused @types/react-gtm-module dev dependency with no active codebase references

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change—removing unused frontend dependencies from package.json.
Description check ✅ Passed The description is directly related to the changeset, providing detailed explanations of which dependencies were removed and why, along with verification steps.
Linked Issues check ✅ Passed The PR successfully addresses all requirements from issue #3142: both rxjs and @types/react-gtm-module have been removed from frontend/package.json with appropriate verification steps completed.
Out of Scope Changes check ✅ Passed All changes are scoped to dependency cleanup in frontend/package.json and pnpm-lock.yaml, with no unrelated modifications introduced.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between dfc2bf0 and bde4d4d.

⛔ Files ignored due to path filters (1)
  • frontend/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (1)
  • frontend/package.json
💤 Files with no reviewable changes (1)
  • frontend/package.json

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 2, 2026

@arkid15r arkid15r enabled auto-merge January 2, 2026 19:06
Copy link
Collaborator

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@arkid15r arkid15r added this pull request to the merge queue Jan 2, 2026
Merged via the queue into OWASP:main with commit 183970c Jan 2, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove unused frontend dependencies (rxjs, @types/react-gtm-module)

2 participants